Skip to content

feat(cli): add step to install ArgoCD#219

Merged
bachgg merged 42 commits intomainfrom
bach/install-argocd
Mar 12, 2026
Merged

feat(cli): add step to install ArgoCD#219
bachgg merged 42 commits intomainfrom
bach/install-argocd

Conversation

@bachgg
Copy link
Contributor

@bachgg bachgg commented Mar 6, 2026

  • oms beta install argocd to install ArgoCD helm chart with desired version
  • One can pass a chart version to --version and if not, the latest chart version is installed
  • Pass --deploy-dc-config to install a local cluster secret for ArgoCD, AppProject resources, secrets for ArgoCD to read manifests from git repo and helm registry
  • Repo & registry secrets are passed into arguments
  • Bootstrap resources after helm install are taken from the charts repo

To manually test this PR:

  • make a kind cluster: kind create cluster
  • install: ./oms beta install argocd --dc-id 7 --git-password 00001112222 --registry-password 12345999 --deploy-dc-config
  • open ArgoCD UI according to command output
  • check git secret: kubectl get secrets/argocd-codesphere-repos-read -nargocd -ojson | jq -r ".data.password" | base64 -d
  • check registry secret: kubectl get secrets/argocd-codesphere-oci-read -nargocd -ojson | jq -r ".data.password" | base64 -d

@bachgg bachgg force-pushed the bach/install-argocd branch from 1e382ed to 3d013d4 Compare March 6, 2026 12:20
@bachgg bachgg requested review from NautiluX and dematteob March 6, 2026 12:52
Copy link
Member

@NautiluX NautiluX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the update and making some artifacts optional!

@bachgg bachgg force-pushed the bach/install-argocd branch from b0f1f0f to fcbd1be Compare March 11, 2026 16:51
@bachgg bachgg requested a review from NautiluX March 12, 2026 10:09
bachgg and others added 2 commits March 12, 2026 10:10
Signed-off-by: bachgg <82871810+bachgg@users.noreply.github.com>
Copy link
Member

@NautiluX NautiluX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only two minor comments, rest lgtm :)

bachgg and others added 3 commits March 12, 2026 15:35
@bachgg bachgg merged commit 590a59b into main Mar 12, 2026
6 checks passed
@bachgg bachgg deleted the bach/install-argocd branch March 12, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants